Fix configuring lib targets with profile options

Fix cargo test --no-run

Fix test for `--test` filtering

The flag now only applies to integration tests, not all targets with the
specified name.

Remove an extraneous is_test condition

If a profile's `test` flag is set to true, then it will always generate a unit
test so the `is_test` check is redundant.

Fix cargo test filtering for binaries

This adds a new `--bin` flag to `cargo test` to specifically say that a binary
should be tested. Additionally the dependencies are tweaked such that binaries
to not depend on themselves being available.

Overhaul how cargo treats profiles

This commit is a complete overhaul of how Cargo handles compilation profiles
internally. The external interface of Cargo is not affected by this change.

Previously each Target had a Profile embedded within it. Conceptually a Target
is an entry in the manifest (a binary, benchmark, etc) and a Profile controlled
various flags (e.g. --test, -C opt-level, etc). Each Package then contained many
profiles for each possible compilation mode. For example a Package would have
one target for testing a library, benchmarking a library, documenting a library,
etc. When it came to building these targets, Cargo would filter out the targets
listed to determine what needed to be built.

This filtering was largely done based off an "environment" represented as a
string. Each mode of compilation got a separate environment string like `"test"`
or `"bench"`. Altogether, however, this approach had a number of drawbacks:

* Examples in release mode do not currently work. This is due to how examples
  are classified and how release mode is handled (e.g. the "release" environment
  where examples are meant to be built in the "test" environment).
* It is not trivial to implement `cargo test --release` today.
* It is not trivial to implement `cargo build --bin foo` where *only* the binary
  `foo` is built. The same is true for examples.
* It is not trivial to add selective building of a number of
  binaries/examples/etc.
* Filtering the list of targets to build has some pretty hokey logic that
  involves pseudo-environments like "doc-all" vs "doc". This logic is duplicated
  in a few places and in general is quite brittle.
* The TOML parser greatly affects compilation due to the time at which profiles
  are generated, which seems somewhat backwards.
* Profiles must be overridden, but only partially, at compile time becuase only
  the top-level package's profile is applied.

In general, this system just needed an overhaul. This commit made a single
change of separating `Profile` from `Target` and then basically hit `make` until
all the tests passed again. The other large architectural changes are:

* Environment strings are now entirely gone.
* Filters are implemented in a much more robust fashion.
* Release mode is now handled much more gracefully.
* The unit of compilation in the backend is no longer (package, target) but
  rather (package, target, profile). This change had to be propagated many
  location in the `cargo_rustc` module.
* The backend does not filter targets to build *at all*. All filtering now
  happens entirely in the frontend.

I'll test issues after this change lands, but the motivation behind this is to
open the door to quickly fixing a number of outstanding issues against Cargo.
This change itself is not intended to close many bugs.

Auto merge of #1433 - alexcrichton:dylib, r=huonw

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Remove dependency on DynamicLibrary

This may not end up being stable for 1.0, and we don't really need much
functionality from it anyway.

Auto merge of #1431 - IvanUkhov:newlines, r=alexcrichton

Hi,

It seems that people generally prefer not having newlines after section headers in their `Cargo.toml`s. However, when one creates a new package via `cargo new`, a newline gets automatically inserted after `[package]`. Then a typical `Cargo.toml` ends up being some what inconsistent: there is a newline after `[package]`, and there are no newlines after other headers.

On top of that, the examples given on crates.io do not seem to be consistent: [sometimes](http://doc.crates.io/index.html) there is a newline after `[package]`, and [sometimes](http://doc.crates.io/manifest.html) there is none. The same applies to other section names.

I am wondering if there should be some style conventions in this regard.

Regards,
Ivan

Auto merge of #1428 - alexcrichton:update, r=alexcrichton

Make the usage of newlines in Cargo.toml consistent

Auto merge of #1429 - IvanUkhov:build-script, r=alexcrichton

Update to rust master

Fix a typo in the description of the links manifest key

Auto merge of #1425 - alexcrichton:issue-1421, r=huonw

This was accidentally broken in a recent refactoring.

Closes #1421

Re-fix the `help` command's implementation

This was accidentally broken in a recent refactoring.

Closes #1421

Auto merge of #1402 - cmr:issue-1015, r=alexcrichton

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Add rustc_private feature to tests which need it.

Add dedicated metadata for adding -l/-L flags

Deprecates the rustc-args metadata keyword in favor of rustc-link-lib and
rustc-link-search, which are more precise and allows for easy, correct
handling of spaces in pathnames.

Closes #1015

Auto merge of #1412 - jakub-:rust-nightly, r=alexcrichton

Update to the latest nightly

Auto merge of #1409 - alexcrichton:less-compiles, r=huonw

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Auto merge of #1406 - alexcrichton:issue-1404, r=wycats

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1410 - johnz133:patch-2, r=huonw

cargo build now outputs to /target/debug, changed doc to reflect that.
steve suggested to add description for cargo build --release
autogenerates to guide.html. Used pre and code tags instead of ``` by following previous examples.

Updated build in guide.md

autogenerates to guide.html

Don't add non-compiled packages to the graph

This was previously required to ensure that all relevant metadata files would
remain in place and not get deleted, but this auto-clean behavior has since been
removed so this should no longer be necessary.

Don't intermingle targets between packages

When loading targets to compile, be sure to use the targets from the package
that's being passed down to the compilation step, not the one that was passed in
which is overridden.

Closes #1404

Auto merge of #1396 - alexcrichton:issue-1390, r=huonw

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Improve experience with the flexible target specification

When calculating the output directory filename, only use the "file name"
component of the target triple specified as otherwise the output could be placed
into an odd location.

Closes #1390

Auto merge of #1395 - alexcrichton:profile, r=brson

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

Auto merge of #1397 - alexcrichton:update, r=alexcrichton

Continuation of #1384

Fix all tests from the update to rust master

Reduce usage of unstable features

Update to rust master

Merge branch 'master' of https://github.com/Victor-Savu/cargo into update

Improve the profiling output of Cargo

* Don't use `{:?}` in profiling output, it's too noisy
* Allow an integer to be specified with `CARGO_PROFILE` indicating how many
  levels deep should be printed.

Auto merge of #1389 - jimmycuadra:patch-1, r=alexcrichton

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.

Document the `harness` manifest option.

Add documentation for the `harness` manifest option, introduced in a6589681351dc4a87b6fc268d47b7d608004b455.

Reverting to rustc nightly 2015-03-04 because threadpool build fails with more recent versions.

Merge branch 'master' of https://github.com/rust-lang/cargo

Updated versions for rustc, curl, docopt and regex.

Auto merge of #1373 - alexcrichton:issue-785, r=wycats

This commit now funnels all output of Cargo by default to be in `target/debug`
instead of the bare `target` directory. This change is targeted at raising
awareness of whether a debug build is being used (as opposed to a release
build). It is also aimed at remedying a common scenario where `cargo build` is
followed by `cargo build --release` and then the debug binaries are run by
accident.

This does not yet explore the option of providing symlinks to the most recent
build, hence this commit is a breaking change due to the restructuring of the
layout of the output.

Note that this commit does **not** change the output location for documentation.
All output of `cargo doc` continues to be funneled into the `target/doc`
directory.

Closes #785

Change the default output location to target/debug

This commit now funnels all output of Cargo by default to be in `target/debug`
instead of the bare `target` directory. This change is targeted at raising
awareness of whether a debug build is being used (as opposed to a release
build). It is also aimed at remedying a common scenario where `cargo build` is
followed by `cargo build --release` and then the debug binaries are run by
accident.

This does not yet explore the option of providing symlinks to the most recent
build, hence this commit is a breaking change due to the restructuring of the
layout of the output.

Note that this commit does **not** change the output location for documentation.
All output of `cargo doc` continues to be funneled into the `target/doc`
directory.

Closes #785

Auto merge of #1387 - alexcrichton:issue-1382, r=brson

A synthetic "Package" is being created as part of this step but its paths were
pointing at the original root, not the unpacked tarball for testing. This meant
that some assumptions about the relative-ness of paths didn't quite end up
working out.

Closes #1382

Correctly set up paths when packaging

A synthetic "Package" is being created as part of this step but its paths were
pointing at the original root, not the unpacked tarball for testing. This meant
that some assumptions about the relative-ness of paths didn't quite end up
working out.

Closes #1382

Auto merge of #1365 - ches:docs, r=brson

While reading through the docs, this section broke my concentration even though the functionality's purpose is fairly obvious, because of a grammatical agreement problem and some redundant/wordy terms. I hope this is an improvement.

A couple of other minor changes:

1. "Limitations" sound like things Cargo is *incapable* of doing. The points here are conscious decisions, so calling them "restrictions" seemed more appropriate to me.
2. TOML's source is linked to on the FAQ page, but I'm pretty likely to land directly on the Configuration page from a search, so a direct link from there seemed helpful.

r? @steveklabnik for docs.

Auto merge of #1383 - alexcrichton:issue-1381, r=huonw

Ensure we don't get erroneous information about invalid log levels or such.

Closes #1381

fixed feature attribute

removed [feature(env)] attributes

Explicitly remove RUST_LOG when crawling output

Ensure we don't get erroneous information about invalid log levels or such.

Closes #1381

Updated to rust nightly 2015-03-04

Auto merge of #1376 - fenhl:patch-1, r=alexcrichton

`dependencie"` → `dependencies`

Fix a typo in Zsh completion

`dependencie"` → `dependencies`

Auto merge of #1372 - alexcrichton:flaky, r=alexcrichton

Due to random hash map traversals this test could build either crate `a` or
crate `b` first, causing the first four output lines to possibly change. Two of
the first four lines are guaranteed to be the compilation of `b` and the other
two are the compilation of `a` as well as building its build script, but no
other lines about compiling `a` can happen until the dependency `b` has finished
compiling.

Relax output restrictions to make a test less flaky

Due to random hash map traversals this test could build either crate `a` or
crate `b` first, causing the first four output lines to possibly change. Two of
the first four lines are guaranteed to be the compilation of `b` and the other
two are the compilation of `a` as well as building its build script, but no
other lines about compiling `a` can happen until the dependency `b` has finished
compiling.

Auto merge of #1370 - alexcrichton:issue-1299, r=huonw

Apparently git submodules are checked in as '/' not '\' so if a '\' leaks
through it'll end up not being found!

Closes #1299

Auto merge of #1371 - alexcrichton:issue-1289, r=huonw

Before this commit the `dep_targets` function of `Context` didn't actually
return all dependencies in the sense that unit tests (and likely examples) would
not have the same package's library listed as a dependency. This commit
rectifies the situation by ensuring that the package's library is included
whenever necessary in the dependency list.

Closes #1289

Fix dep_targets for tests depending on libraries

Before this commit the `dep_targets` function of `Context` didn't actually
return all dependencies in the sense that unit tests (and likely examples) would
not have the same package's library listed as a dependency. This commit
rectifies the situation by ensuring that the package's library is included
whenever necessary in the dependency list.

Closes #1289

Translate '\' to '/' for submodule names

Apparently git submodules are checked in as '/' not '\' so if a '\' leaks
through it'll end up not being found!

Closes #1299

Auto merge of #1368 - cyndis:lto-dyn, r=alexcrichton

When building host code, -C prefer-dynamic is passed, but that option
is mutually exclusive with -C lto. Thus when LTO is specified, rustc
errors out.

Add test for having lto enabled while having a build script

Disable LTO for host (plugin, build script) builds

When building host code, -C prefer-dynamic is passed, but that option
is mutually exclusive with -C lto. Thus when LTO is specified, rustc
errors out.

Auto merge of #1367 - alexcrichton:fix-tar, r=alexcrichton

Fixes a bug reported in alexcrichton/flate2-rs#15 where the header wasn't being
properly read.

Update tar dependency

Fixes a bug reported in alexcrichton/flate2-rs#15 where the header wasn't being
properly read.

docs: Improve grammar/wording for `cargo owner`

Auto merge of #1360 - alexcrichton:new-io, r=alexcrichton

cc @aturon, this is one giant commit.

Update to rust master and std::{io, path}

Auto merge of #1354 - mbrubeck:gh-pages-title, r=alexcrichton

This should make them easier to find via search engines, or in browser history/bookmarks.

See #1302 for an old version of this PR.

Add "Cargo" to doc titles

Auto merge of #1347 - alexcrichton:no-more-urllib, r=huonw

We've had too many problems using urllib to make it worth it.

Closes #1346
Closes #1088
cc #1319 (using a proxy through curl)

Merge pull request #1350 from sfackler/master

Don't time out after 0.006 seconds

Don't time out after 0.006 seconds

Fixup some python style which was previously reverted

Probe for `curl` when configuring

Revert "Directly use python’s urllib instead of curl in etc/dl-snapshot.py"

This reverts commit aceba880fdf83cbd694fa1a6e572c600b17d1c37.

Conflicts:
src/etc/dl-snapshot.py

Auto merge of #1345 - badboy:pass-cargo-name, r=alexcrichton

This reverts a change from ee5e24ff8b5.

Before it passed "foo" as `argv[0]`, which does not help much (consider the case there is one script symlinked to different names to handle different things).

This patch changes that to `cargo-{command}`.

It also reverts the passing of "foo" on the help command. Before it used `os::arg`, a different (and ugly) solution would be:

    let args = &[env::args().next().unwrap().clone(), "-h".to_string()];

Strip first unused argument when passing to subcommand

Auto merge of #1348 - alexcrichton:rustup, r=alexcrichton

Fixup warnings and the travis script

Merge branch 'rustup' of https://github.com/globin/cargo into rustup

Auto merge of #1344 - lfairy:dehyphenator, r=alexcrichton

Closes #1342.

Auto merge of #1291 - alexcrichton:issue-1083, r=huonw

Hopefully this will help diagnose #1083

fix style

fix travis script

rustup

Use an underscore instead of a hyphen as a dummy crate name

Closes #1342.

Auto merge of #1340 - alexcrichton:issue-1329, r=huonw

Closes #1329

Auto merge of #1341 - alexcrichton:issue-1333, r=huonw

Closes #1333

Bump default timeout to 60s

Closes #1333

Download registry packages on `cargo fetch`

Closes #1329

Auto merge of #1339 - jakub-:patch-1, r=alexcrichton

Add `Third-party cargo subcommands` to the README

Auto merge of #1321 - dlevy47:bugfix/no-proxy, r=alexcrichton

Currently, cargo uses `curl_easy_setopt` to explicitly set the http proxy on a curl handle if any of the following are present:

* cargo config `http.proxy`
* git config `http.proxy`
* `HTTP_PROXY` environment variable

The act of explicitly setting the http proxy disables curl's logic for determining which proxy to use for a URL, and breaks the `no_proxy` environment variable. This PR privatizes `cargo::ops::registry::http_proxy` and adds an additional function `cargo::ops::registry::http_proxy_exists` to determine whether or not to use the `git2-curl` subtransport.

Change cargo to explicitly set curl proxy settings only when necessary
(when it is specified in either the cargo config or the git config).
Otherwise, use curl's logic to pick a proxy (or not).

Auto merge of #1288 - alexcrichton:issue-1058, r=huonw

Previously the build script's profile was used instead of the target's build
script, causing the wrong environment variables to get set.

Closes #1058

Set build script env vars correctly

Previously the build script's profile was used instead of the target's build
script, causing the wrong environment variables to get set.

Closes #1058

Auto merge of #1314 - AnthIste:build-script-updates, r=alexcrichton

The examples given for `build.rs` do not compile against the latest rust. Instead of adapting to use modules such as `std::old_io`, I attempted to port the examples to use the new `io` and `path` modules.

Update example build.rs scripts to reflect latest Rust changes

Auto merge of #1317 - alexcrichton:issue-1297, r=huonw

Not sure why we were doing this, it's quite useful!

Closes #1297

Don't discard docopt error information

Not sure why we were doing this, it's quite useful!

Closes #1297